Skip to content

ARROW-11336: [C++][Doc] Improve Developing on Windows docs#9855

Closed
ianmcook wants to merge 11 commits into
apache:masterfrom
ianmcook:ARROW-11336
Closed

ARROW-11336: [C++][Doc] Improve Developing on Windows docs#9855
ianmcook wants to merge 11 commits into
apache:masterfrom
ianmcook:ARROW-11336

Conversation

@ianmcook

@ianmcook ianmcook commented Mar 30, 2021

Copy link
Copy Markdown
Member
  • Improves the Developing on Windows docs
    • Adds instructions for Visual Studio 2019 users
    • Documents how to use vcpkg to install Arrow dependencies on Windows
    • Improves some ambiguous instructions and fixes minor errors
  • Adds more mentions of vcpkg to the Building Arrow C++ docs
  • Describes how to open the Visual Studio IDE with a conda environment activated (ARROW-11192)

@github-actions

Copy link
Copy Markdown

@ianmcook ianmcook changed the title ARROW-11336: [C++][Doc] Improve Developing on Windows docs [WIP] ARROW-11336: [C++][Doc] Improve Developing on Windows docs Mar 31, 2021
@ianmcook ianmcook marked this pull request as ready for review March 31, 2021 15:59
@ianmcook

Copy link
Copy Markdown
Member Author

@bkietz please review

@jorisvandenbossche jorisvandenbossche left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't use windows, so can't comment on the content ;) But was just reading through (looks good!), and have one small sphinx suggestion

Comment thread docs/source/developers/cpp/windows.rst Outdated

@pitrou pitrou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good generally, two small comments.

Comment thread docs/source/developers/cpp/windows.rst Outdated
Comment thread docs/source/developers/cpp/windows.rst Outdated

@bkietz bkietz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor comments, otherwise this looks good

Comment thread docs/source/developers/cpp/windows.rst Outdated
Comment thread docs/source/developers/cpp/windows.rst Outdated
Comment thread docs/source/developers/cpp/windows.rst Outdated
@ianmcook ianmcook requested a review from bkietz April 7, 2021 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants